-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Storage Error Codes for Copies #45394
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Storage Error Codes for Copies #45394
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looking good so far - you should definitely make the change to the queue pom file though so the pipeline can get further into the analyze step :)
...e/azure-storage-common/src/main/java/com/azure/storage/common/policy/RequestRetryPolicy.java
Outdated
Show resolved
Hide resolved
...ure-storage-common/src/test/java/com/azure/storage/common/policy/RequestRetryPolicyTest.java
Show resolved
Hide resolved
...e/azure-storage-blob/src/test/java/com/azure/storage/blob/specialized/BlockBlobApiTests.java
Show resolved
Hide resolved
...re-storage-blob/src/test/java/com/azure/storage/blob/specialized/BlockBlobAsyncApiTests.java
Show resolved
Hide resolved
...e/azure-storage-common/src/main/java/com/azure/storage/common/policy/RequestRetryPolicy.java
Outdated
Show resolved
Hide resolved
@@ -16,7 +16,7 @@ autorest | |||
### Code generation settings | |||
``` yaml | |||
use: '@autorest/java@4.1.42' | |||
input-file: https://raw.githubusercontent.com/Azure/azure-rest-api-specs/30318bc99a721df4ebdfdbf40696739ede75564f/specification/storage/data-plane/Microsoft.BlobStorage/stable/2025-07-05/blob.json | |||
input-file: https://raw.githubusercontent.com/nickliu-msft/azure-rest-api-specs/39829afa70461186963adab664aebd3d9c22971a/specification/storage/data-plane/Microsoft.BlobStorage/stable/2025-11-05/blob.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
once the specs are merged into main, we should change this to point to azure instead of nick's fork - same for the file share readme
...e/azure-storage-common/src/main/java/com/azure/storage/common/policy/RequestRetryPolicy.java
Outdated
Show resolved
Hide resolved
API Change CheckAPIView identified API level changes in this PR and created the following API reviews com.azure:azure-storage-blob |
...e/azure-storage-common/src/main/java/com/azure/storage/common/policy/RequestRetryPolicy.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good - the queue test issue with CI needs to be fixed at some point, but that can be resolved after the API views are generated.
822e644
into
feature/storage/stg99base
Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines